Skip to content

fix(respect): issue with json encoding for custom content-type - #2490

Merged
DmitryAnansky merged 4 commits into
mainfrom
fix/custom-application-json-encoded-headers
Jan 8, 2026
Merged

fix(respect): issue with json encoding for custom content-type#2490
DmitryAnansky merged 4 commits into
mainfrom
fix/custom-application-json-encoded-headers

Conversation

@DmitryAnansky

@DmitryAnansky DmitryAnansky commented Jan 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

What/Why/How?

Corrected an issue where Respect did not properly JSON-encode request bodies for custom content-types containing numbers (e.g. application/ast.some.test.value-3v3.info+json).

Reference

Testing

Screenshots (optional)

Check yourself

  • Code changed? - Tested with Redoc/Realm/Reunite (internal)
  • All new/updated code is covered by tests
  • New package installed? - Tested in different environments (browser/node)
  • Documentation update considered

Security

  • The security impact of the change has been considered
  • Code follows company security practices and guidelines

@changeset-bot

changeset-bot Bot commented Jan 8, 2026 •
edited
Loading

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: f5c183e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@redocly/respect-core Patch
@redocly/cli Patch
@redocly/openapi-core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@DmitryAnansky DmitryAnansky self-assigned this Jan 8, 2026
@github-actions

github-actions Bot commented Jan 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 78.58% (🎯 78%) 6190 / 7877
🔵 Statements 78% (🎯 78%) 6396 / 8199
🔵 Functions 81.95% (🎯 81%) 1263 / 1541
🔵 Branches 70.41% (🎯 70%) 4246 / 6030
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/respect-core/src/utils/api-fetcher.ts 54.79% 45.76% 73.33% 54.48% 87-93, 123, 149-161, 170-171, 183-186, 191-194, 199, 202-223, 236-249, 257, 304-310, 317-395, 403, 411, 432, 447, 454, 457, 461
Generated in workflow #8291 for commit f5c183e by the Vitest Coverage Report Action

@github-actions

github-actions Bot commented Jan 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
CLI Version Mean Time ± Std Dev (s) Relative Performance (Lower is Faster)
cli-latest 3.454s ± 0.051s ▓ 1.01x
cli-next 3.430s ± 0.063s ▓ 1.00x (Fastest)

@DmitryAnansky
DmitryAnansky marked this pull request as ready for review January 8, 2026 17:59
@DmitryAnansky
DmitryAnansky requested review from a team as code owners January 8, 2026 17:59
@DmitryAnansky
DmitryAnansky merged commit 1ee0521 into main Jan 8, 2026
40 checks passed
@DmitryAnansky
DmitryAnansky deleted the fix/custom-application-json-encoded-headers branch January 8, 2026 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adamaltman adamaltman adamaltman approved these changes
@tatomyr tatomyr tatomyr approved these changes

Assignees

@DmitryAnansky DmitryAnansky

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@DmitryAnansky @adamaltman @tatomyr